-
Notifications
You must be signed in to change notification settings - Fork 51
GUI Redesign Incorporation into Master #696
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
GUI Redesign Point Initialization Page
GUI Redesign Weather Model Page Refactor
GUI Redesign Initial Ninja Com Integration
…anding and shrinking mainwindow
…nings. seems the duplicate "aboutQtAction" and "exportSolutionAction" were introduced in commit/merge 787ef5e, one of the extra "verticalLayout_26" was added in commit/merge 5f5cc8e, and the additional "verticalLayout_" "_26", "_29", and "_31" were added in commit/merge 006911f. It is confusing though, because some of the later commits/merges make it look like these same changes were added then as well, but I double checked and these commits seem to be the earliest that such additions were found in the raw files of the mainwindow.ui file code. Not detecting any differences in code behavior, so I think that these should be the only quirky changes accidentally introduced during the last few merges.
…nd point radius should update alongside bounding box coordinates, for #648
GUI Redesigns Surface Input Improvements for DEM dowloading and reading
GUI redesign weather model initial work
it technically works, but there's some weird final extra flush-like emit that is occurring after the process is done, leading to the final message and close being replaced with another message
GUI Redesign Map Layer Tree Button
…function, also an improvement to the organization of the domainAvgTable checkRows() app state checking methods. Should be a lot clearer now. For issue #687
…d, moved the state checking to appState, now the icons and tooltips are getting updated accordingly. For issue #687
…, to greying/ungreying. For issue #687
GUI Redesign Domain Avg Table
…from commit 7d8c075, if there are differences between qt4 and qt6 GUI results, it is something else
…to improve inputWindHeight combo boxes while I was at it.
… a bit better, and make it so you don't have to click into the first text box to edit it.
| <link rel="stylesheet" href="leaflet/L.Control.Layers.Tree.css" /> | ||
|
|
||
| <!-- Leaflet and Draw dependencies --> | ||
| <script src="https://cdnjs.cloudflare.com/ajax/libs/jszip/3.10.1/jszip.min.js"></script> |
Check warning
Code scanning / CodeQL
Inclusion of functionality from an untrusted source Medium
| <!-- Leaflet and Draw dependencies --> | ||
| <script src="https://cdnjs.cloudflare.com/ajax/libs/jszip/3.10.1/jszip.min.js"></script> | ||
|
|
||
| <script src="https://cdnjs.cloudflare.com/ajax/libs/jszip/3.10.1/jszip.min.js"></script> |
Check warning
Code scanning / CodeQL
Inclusion of functionality from an untrusted source Medium
|
As far as I can tell, it definitely looks good. Not sure if we should try to address the two security things in map.html before the merge, or after the merge. But the overall merge appears to have everything correct. |
The GUI Redesign is at an appropriate stage where we need to merge with the master branch. This will allow users to use the updated GUI as needed and provide us developers with up to date ninja code.